Skip to content

Consolidating track-related logs#677

Merged
alexlivekit merged 2 commits into
mainfrom
alexmigolin/tel-545-investigate-overall-log-volume
May 12, 2026
Merged

Consolidating track-related logs#677
alexlivekit merged 2 commits into
mainfrom
alexmigolin/tel-545-investigate-overall-log-volume

Conversation

@alexlivekit
Copy link
Copy Markdown
Contributor

No description provided.

@alexlivekit alexlivekit requested a review from a team as a code owner May 6, 2026 17:33
Copy link
Copy Markdown
Contributor Author

@alexlivekit alexlivekit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Added subscribedAt attribute to per-track logging.
  • Added closedAt attribute for the "track closed" message.
  • Dropped "mixing track". We can now get this info from "track closed". Forthermore, we already have the library-emitted "track subscribed", which we should ideally drop as well.
  • Left the OnTrackSubscribed as a stack function for the sake of review, but this can now be easily moved to a dedicated function if we wanted to.

@davidzhao davidzhao changed the title Consolidating tarck-related logs Consolidating track-related logs May 6, 2026
Copy link
Copy Markdown
Contributor

@hechen-eng hechen-eng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rebase and fix the CI?

@alexlivekit alexlivekit force-pushed the alexmigolin/tel-545-investigate-overall-log-volume branch 2 times, most recently from a5d78e6 to 3c969bc Compare May 12, 2026 20:00
@alexlivekit alexlivekit force-pushed the alexmigolin/tel-545-investigate-overall-log-volume branch from 3c969bc to 99b318e Compare May 12, 2026 21:19
@alexlivekit alexlivekit merged commit 36d7d59 into main May 12, 2026
5 checks passed
@alexlivekit alexlivekit deleted the alexmigolin/tel-545-investigate-overall-log-volume branch May 12, 2026 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants